chore(deps): bump all @constructive-io/* and graphile-* packages to latest#29
Merged
pyramation merged 3 commits intomainfrom Apr 18, 2026
Merged
Conversation
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
…atest - graphile-build/-pg/-config/-utils/postgraphile: 5.0.0-rc.* -> 5.0.x (stable graduation) - @constructive-io/graphql-test: 4.9.10 -> 4.11.1 - @constructive-io/graphql-codegen: 4.30.0 -> 4.30.1 - @constructive-io/graphql-query: 3.12.11 -> 3.14.1 - @constructive-io/graphql-types: 3.4.3 -> 3.5.0 - @constructive-io/node: 0.10.1 -> 0.10.2 (across provision/rag/worker) - graphile-settings: 4.18.5/4.21.0 -> 4.21.1 (aligned across workspaces) - graphile-test: 4.7.6 -> 4.9.0 - pgsql-test: 4.7.6 -> 4.9.0 All package.json dep bumps only; no code changes. pnpm install clean, existing tests (including the new unified-spatial-combined test) still pass.
…blish tsc passes The generated embedder.ts typed `where`/`data` as `Record<string, unknown>`, which made every call site (`findManyArgs.where = await autoEmbedWhere(...)`, `await autoEmbedInput(cleanedData, ...)`) error under `prepublishOnly` / `prepack` strict `tsc` because `*Filter` and `*Patch` types have no string index signature. CI passes via `ts-jest` (which is lenient about excess-property / index-signature checks), but `makage build` on publish is strict and fails on all 38 generated CLI command files. Make the two helpers generic (`<T extends object>(where: T, ...): Promise<T>`) and cast to `Record<string, unknown>` internally for the field-name indexing. Preserves the runtime behavior (mutation in place, same return semantics) and eliminates the 74 TS2345 errors in `generated/cli/commands/*.ts`. The long-term fix belongs in the upstream `@constructive-io/graphql-codegen` embedder.ts template; filed as follow-up.
f6a9249 to
2c465a1
Compare
4 tasks
devin-ai-integration bot
pushed a commit
to constructive-io/constructive
that referenced
this pull request
Apr 18, 2026
…der template
The CLI codegen emits calls like `findManyArgs.where = await autoEmbedWhere(findManyArgs.where ?? {}, ['embedding'], embedder)` and `await autoEmbedInput(cleanedData, ['embedding'], embedder)` where the first argument is a concrete `*Filter` / `*Patch` type produced by `input-types.ts`. Those generated types don't have a string index signature, so under strict `tsc` every call site errors with:
error TS2345: Argument of type 'XxxFilter' is not assignable to
parameter of type 'Record<string, unknown>'. Index signature for
type 'string' is missing in type 'XxxFilter'.
CI for consumers (agentic-db et al.) passes via `ts-jest`, which is lenient about index-signature / excess-property checks, but `makage build` on publish is strict and fails across all generated `commands/*.ts` files.
Make both helpers generic (`<T extends object>(x: T, ...): Promise<T>`) and cast to `Record<string, unknown>` internally for the field-name indexing. Runtime behavior is unchanged (mutation in place, same return semantics), and call sites now type-check as-is without any generator change.
Consumer-side point fix already landed in constructive-io/agentic-db#29; this PR fixes the upstream template so the next `pnpm generate:all` in any consumer emits a correctly-typed `embedder.ts`.
This was referenced Apr 18, 2026
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Pure dependency bump across the workspace, plus a small CLI codegen runtime fix that is required to unblock
lerna publish/prepackon the@agentic-db/clipackage. No other code changes — onlypackage.jsonspecifiers, the resultingpnpm-lock.yamlchurn, andsdk/cli/generated/cli/embedder.ts.Versions moved:
graphile-build5.0.0-rc.65.0.0graphile-build-pg5.0.0-rc.85.0.1graphile-config1.0.0-rc.61.0.1graphile-utils5.0.0-rc.85.0.1postgraphile5.0.0-rc.105.0.1@constructive-io/graphql-test^4.9.10^4.11.1@constructive-io/graphql-codegen4.30.04.30.1@constructive-io/graphql-query3.12.113.14.1@constructive-io/graphql-types3.4.33.5.0@constructive-io/node^0.10.0/^0.10.1^0.10.2(provision, rag, worker)graphile-settings4.18.5/4.21.04.21.1(aligned across packages/agentic-db + integration-tests)graphile-test^4.7.6^4.9.0pgsql-test^4.7.6^4.9.0(root, packages/agentic-db, integration-tests, cli-e2e-tests, agentic-db-services)Headliner: the
graphile-*andpostgraphilefamily finally graduates from RC tags to stable (5.0.0 / 5.0.1 / 1.0.1). Pinned in the root pnpmoverridesblock so transitive resolution collapses onto one copy.Additional fix — unblock
prepacktsc on@agentic-db/cliRunning
lerna publishfailed on@agentic-db/cli'sprepack(makage build→tsc) with 74 TS2345 errors across all 37 generatedgenerated/cli/commands/*.tsfiles, e.g.:The upstream CLI codegen (
@constructive-io/graphql-codegen@4.30.1) emits calls likeautoEmbedWhere(findManyArgs.where ?? {}, [...], embedder)andautoEmbedInput(cleanedData, [...], embedder)where the first argument is a concrete*Filter/*Patchtype, but the runtime helpers were typed to acceptRecord<string, unknown>. Generated filter/patch types don't have a string indexsignature, so every call site errored under strict
tsc. CI is green because theorm-tests/cli-e2e-tests/rag-testsjobs go throughts-jest, which islenient about index-signature / excess-property checks — only the publish build
runs strict
tsc.Fix in
sdk/cli/generated/cli/embedder.ts: make both helpers generic and cast toRecord<string, unknown>internally for field-name indexing. Runtime behavior isunchanged (mutation in place, same return semantics).
sdk/clinow builds cleanly (tscCJS + ESM), andsdk/sdkis unaffected. Thelong-term home for this fix is the upstream
@constructive-io/graphql-codegenembedder.tstemplate (graphql/codegen/src/core/codegen/templates/embedder.ts);I'll file a follow-up PR against
constructive-io/constructiveso the nextregen replaces this patched file with an equivalently-typed one.
Updates since last revision
main(PR docs+test(readme): combined unifiedSearch + nearbyPlaces ORM example backed by a real integration test #28 landed); regeneratedpnpm-lock.yamlfrom the mergedpackage.jsonspecifiers.autoEmbedWhere/autoEmbedInputgeneric fix above solerna publishcan completeprepackon@agentic-db/cli.Review & Testing Checklist for Human
pnpm-lock.yamldiff is large because almost everygraphile-config/graphile-buildcoordinate in transitive peer hashes re-keyed. Spot-check that the lockfile has exactly one resolved version per package (no duplicate RC + stable copies). Searching for5.0.0-rcshould return zero hits outside of thegrafast/@dataplan/*/pg-sql2/tamedeviloverrides that we're deliberately still pinning to RC.pgsql-testconsumers are now on^4.9.0and bothgraphile-settingsconsumers are on4.21.1, but worth eyeballing in case one was missed.pnpm buildTypeScript errors inpackages/provisionare NOT caused by this PR. Confirmed by stashing the bumps and re-running build on the same commit — identical errors ('ref'/'table_ref'/'source_ref'not onBlueprintTable/BlueprintIndex/BlueprintRelation). It's a drift betweenpackages/provision/src/schemas/*.tsand thenode-type-registry@0.16.0type definitions. Separate cleanup; please don't block this PR on it.sdk/cli/generated/cli/embedder.ts) — nextpnpm generate:allwill overwrite this unless the upstream template is fixed first. Tracked as a follow-up inconstructive-io/constructive.Test plan
cd sdk/cli && pnpm build— reproduces the 74 TS2345 errors before the fix; compiles cleanly after.pnpm install && pnpm -C packages/agentic-db exec jest— the unified-search + spatial tests are the thinnest layer over the bumped plugin stack.Notes
@constructive-io/node@0.10.2is just a re-export of@constructive-io/sdk@0.14.2; same public surface.grafast/@dataplan/json/@dataplan/pg/pg-sql2/tamedeviloverrides — they're still on RC tags because that's what the rest of the ecosystem is pinning.graphile-workerwas already on^0.16.6(latest); no change needed.node-type-registrywas already on^0.16.0(latest); no change needed.Link to Devin session: https://app.devin.ai/sessions/fabecb6a46504fc293feca22d9cc91d4
Requested by: @pyramation